Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Adding SQL Execution + SQL Connector #37

Closed
wants to merge 10 commits into from
Closed

Adding SQL Execution + SQL Connector #37

wants to merge 10 commits into from

Conversation

zacdav-db
Copy link
Contributor

Lot of changes:

  • README updated
  • DESCRIPTION updated
  • Helpers to determine behaviour when on Databricks
  • Databricks SQL Execution API added
  • Datbricks SQL Connector (Python) bindings added via {reticulate}

Zac Davies added 9 commits November 3, 2023 17:53
adding sql execution api functions
e0e8b8a
remove presigned url
38428a0
removing client code
a9b2654
changes:
7566b7f 
 - sql execution functions renamed
 - updated DESCRIPTION (version, libs)
 - README updated
 - Request helpers improved
 - SQL connector added via reticulate
 - onLoad behaviour added
Updating DESCRIPTION
9320776
Changes:
1755d5e 
 - improved detection of env on Databricks for sql connector
 - Arrow imports
Trying to improve db_host() to work when https:// is omitted.
5c43442
Changes:
1591454 
 - fix DESCRIPTION
 - add examples
 - update README
 - rename `detect_brickster_venv` to be `determine...`
 - Update auth + add tests
Lots of docs updates
39f3019
@codecov-commenter
Copy link

codecov-commenter commented Nov 6, 2023 •
edited
Loading

Codecov Report

Merging #37 (a4d6927) into main (6e7a3fa) will decrease coverage by 0.20%.
The diff coverage is 7.85%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@            Coverage Diff             @@
##             main      #37      +/-   ##
==========================================
- Coverage   11.47%   11.27%   -0.20%     
==========================================
  Files          23       26       +3     
  Lines        2746     2882     +136     
==========================================
+ Hits          315      325      +10     
- Misses       2431     2557     +126
Files Coverage Δ
R/databricks-helpers.R 100.00% <100.00%> (ø)
R/package-auth.R 50.00% <100.00%> (+5.81%) ⬆️
R/request-helpers.R 0.00% <0.00%> (ø)
R/sql-query-execution.R 0.00% <0.00%> (ø)
R/sql-connector.R 0.00% <0.00%> (ø)

📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!

@zacdav-db zacdav-db closed this May 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@zacdav-db @codecov-commenter